Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEAT] 회원의 데일리 루틴 목록 조회 #278

Merged
merged 20 commits into from
Jun 21, 2024

Conversation

Chan531
Copy link
Contributor

@Chan531 Chan531 commented Jun 18, 2024

✨ Related Issue

📝 기능 구현 명세

image

  • 회원의 데일리 루틴 목록 조회 api

image

  • 회원의 데일리 루틴 목록 조회 api 테스트

🐥 추가적인 언급 사항

  • 최대한 controller 단에서만 v2를 작성하려고 했는데 그게 여의치 않아서 service 단에서 v2 관련 메소드와 dto가 꽤 추가되었습니다.
  • 맞는 방식인지 궁금합니다.
  • 현재 루틴은 가나다순으로 정렬이 된 상태지만, 테마는 무지개순으로 정렬되지 않은 상태입니다.
  • 테마를 어떻게 해야 무지개순으로 정렬해야할지 고민 중입니다... (꽤나 까다로움)
  • 현재까지 테마를 무지개순으로 정렬하는 방안은 다음과 같습니다
  1. E로 시작하는 색이 앞에 온다.
  2. 16진수의 두번째 문자를 15로 나눈 나머지가 작을 수록 앞에 온다. 만약 같을 경우 그 수가 작은 것이 앞에 온다.
    (ex. E19098과 EF596F는 나머지가 같기 때문에 전자가 앞에 온다.
    EDBF4F과 EB9D07는 후자가 앞에 온다.)
  3. E로 시작하지 않는다면 사전순으로 작을 수록 앞에 온다.
    (ex. 52A1DA는 6392D8보다 앞에 온다.)
  • 이렇게까지 해야하나 싶어서 일단 테마 무지개순 정렬은 보류했습니다.

@Chan531 Chan531 added chan 찬 작업 feat 기능 개발 test 테스트 labels Jun 18, 2024
@Chan531 Chan531 requested a review from thguss June 18, 2024 08:35
@Chan531 Chan531 self-assigned this Jun 18, 2024
Copy link
Member

@thguss thguss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다.
서비스 계층에서 버저닝만 제거하면 좋을 것 같습니다.
추가로, 작성한 리뷰는 작성자가 직접 resolve하는 것이 어떨까요!? 작성한 리뷰가 반영되었는지 다시 확인해야하는데 이미 resolve 되어 있으면 진행이 불편해지는 감이 있어 요청 드립니다 :)

Copy link
Member

@thguss thguss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다 👍
의논이 필요해보이는 사항이 있어 comment로 남깁니다!
리뷰 또는 소통 채널에서 의견 공유하면 좋을 것 같습니다~

@Chan531 Chan531 requested a review from thguss June 19, 2024 04:59
@Chan531 Chan531 requested a review from thguss June 19, 2024 08:15
@Chan531 Chan531 requested a review from thguss June 19, 2024 11:59
Copy link
Member

@thguss thguss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다~ (정말루.. 😭😭)
DTO 이름 외 별 다른 이슈는 없어보여 우선 승인해두겠습니다.

@Chan531 Chan531 requested a review from thguss June 21, 2024 17:11
@Chan531 Chan531 merged commit e74765f into develop Jun 21, 2024
1 check passed
@Chan531 Chan531 deleted the feat/#273-member-daily-routine branch June 21, 2024 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chan 찬 작업 feat 기능 개발 test 테스트
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 회원의 데일리 루틴 목록 조회
2 participants